Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[GHSA-cmh9-rx85-xj38] sidekiq-unique-jobs UI server vulnerable to XSS & RCE in Redis #3546

Conversation

Earlopain
Copy link

@Earlopain Earlopain commented Feb 15, 2024

Updates

  • Affected products
  • CVSS
  • Severity
  • Summary

Comments
The CVE analysis of this vulnerability was assigned a score of 7.1. This is quite different to the 10 on initial report.

This updates the score of this advisory to what is displayed at https://nvd.nist.gov/vuln/detail/CVE-2024-25122

I have also removed the mention of RCE in the title. There was no RCE vulnerability. See mhenrixon/sidekiq-unique-jobs#833 (comment) and mhenrixon/sidekiq-unique-jobs#833 (comment)

Finally, I added a lower bound to vulnerable versions, it was only introduced in v6.0.0.rc7, see https://github.com/mhenrixon/sidekiq-unique-jobs/blob/main/CHANGELOG.md#v600rc7-2018-07-23

The advisory in the repository has already been updated: GHSA-cmh9-rx85-xj38

@github
Copy link
Collaborator

github commented Feb 15, 2024

Hi there @pboling and @mhenrixon! A community member has suggested an improvement to your security advisory. If approved, this change will affect the global advisory listed at github.com/advisories. It will not affect the version listed in your project repository.

This change will be reviewed by our Security Curation Team. If you have thoughts or feedback, please share them in a comment here! If this PR has already been closed, you can start a new community contribution for this advisory

@github-actions github-actions bot changed the base branch from main to Earlopain/advisory-improvement-3546 February 15, 2024 18:08
Copy link

@mhenrixon mhenrixon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

@pboling
Copy link

pboling commented Feb 16, 2024

I don't understand how it wasn't already fixed, since I had already edited the advisory to update these things. Too many sources of truth!

@github
Copy link
Collaborator

github commented Feb 16, 2024

Hi there @pboling and @mhenrixon! A community member has suggested an improvement to your security advisory. If approved, this change will affect the global advisory listed at github.com/advisories. It will not affect the version listed in your project repository.

This change will be reviewed by our Security Curation Team. If you have thoughts or feedback, please share them in a comment here! If this PR has already been closed, you can start a new community contribution for this advisory

@github
Copy link
Collaborator

github commented Feb 16, 2024

Hi there @pboling and @mhenrixon! A community member has suggested an improvement to your security advisory. If approved, this change will affect the global advisory listed at github.com/advisories. It will not affect the version listed in your project repository.

This change will be reviewed by our Security Curation Team. If you have thoughts or feedback, please share them in a comment here! If this PR has already been closed, you can start a new community contribution for this advisory

@Earlopain
Copy link
Author

Sorry for the double-ping, I wanted to add a lower bounds to the affected versions since there wasn't always a Web UI. It was only introduced in v6.0.0.rc7, see https://github.com/mhenrixon/sidekiq-unique-jobs/blob/main/CHANGELOG.md#v600rc7-2018-07-23

I expected another modification to create an additional PR but it seems to just have eaten my previous changes.

Copy link

@pboling pboling left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks for noticing the discrepancy, and for doing the research on the lower bound!

@advisory-database advisory-database bot merged commit 3672343 into Earlopain/advisory-improvement-3546 Feb 20, 2024
1 check passed
@advisory-database advisory-database bot deleted the Earlopain-GHSA-cmh9-rx85-xj38 branch February 20, 2024 16:40
@advisory-database
Copy link
Contributor

Hi @Earlopain! Thank you so much for contributing to the GitHub Advisory Database. This database is free, open, and accessible to all, and it's people like you who make it great. Thanks for choosing to help others. We hope you send in more contributions in the future!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants